Skip to content

People 3rd review update#114

Merged
froggy1014 merged 4 commits intodevelopfrom
feat/synchronize-excel
Mar 8, 2025
Merged

People 3rd review update#114
froggy1014 merged 4 commits intodevelopfrom
feat/synchronize-excel

Conversation

@froggy1014
Copy link
Contributor

@froggy1014 froggy1014 commented Mar 7, 2025

Description

기존에 Excel Sheet에 데이터 적재해서 사용하고 있어서 제가 크게 건들인건 없었고, 후기 수집된걸 넣는 과정이 수동이더군요;

일단 3기 회원들 후기 수집된거 업데이트 완료했습니다.

그런데, sorting 함수가 좀 잘못된거 수정해서 PR 올렸어요. 밑에 Vercel Preview로 확인해주세요

홈페이지 관련된 정적데이터들은 빌드타임때, 엑셀에서 가져와 정제되어 .json으로 생성이 되고 있어요.

"build": "yarn run script && next build",

AS-IS TO-BE
image image

사진을 추가하니까, Vercel Free Tier에서 이미지 최적화 허용량 초과가되어서 아래와 같은 현상이 발생해서,
next.config.js에서 이미지 최적화 옵션 잠시 꺼두겠습니다.

image

@vercel
Copy link

vercel bot commented Mar 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sipe-team ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2025 0:25am

@froggy1014 froggy1014 merged commit c941166 into develop Mar 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant